Skip to content

Comments

feat: Support Elixir 1.18#2

Merged
ymtszw merged 8 commits intomainfrom
feat/support_elixir_1.18_yL3I3YwE
Jan 31, 2025
Merged

feat: Support Elixir 1.18#2
ymtszw merged 8 commits intomainfrom
feat/support_elixir_1.18_yL3I3YwE

Conversation

@ymtszw
Copy link
Member

@ymtszw ymtszw commented Jan 30, 2025

  • test: guard in assert_match/2
  • chore: use mise and direnv for development
  • chore: Elixir 1.18 drops ~R//
  • ci: drop Erlang 23, support 27

test "should work with guards" do
assert_match(%{key: 1}, map when is_map_key(map, :key))

# guard works with assert_match/2 but has limitation: bindings inside patterns cannot be used from outside
Copy link
Contributor

@KyosukeUryu KyosukeUryu Jan 31, 2025

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

= match演算子ではassertionと同時に束縛できたがmatch?/2を利用するmatchだと束縛はできないのは確かに制限事項となりますね 📝

Copy link
Contributor

@KyosukeUryu KyosukeUryu left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM 🔥

@ymtszw
Copy link
Member Author

ymtszw commented Jan 31, 2025

Mergeしてtag打ちます

@ymtszw ymtszw merged commit 5f4efca into main Jan 31, 2025
12 checks passed
@ymtszw ymtszw deleted the feat/support_elixir_1.18_yL3I3YwE branch January 31, 2025 04:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants